Skip to content

[O2B-1539] Add index on QC flags to help GAQ summary computation#2099

Merged
isaachilly merged 2 commits intomainfrom
improv/O2B-1539/Add-index-to-speed-up-GAQ-computation
Mar 6, 2026
Merged

[O2B-1539] Add index on QC flags to help GAQ summary computation#2099
isaachilly merged 2 commits intomainfrom
improv/O2B-1539/Add-index-to-speed-up-GAQ-computation

Conversation

@isaachilly
Copy link
Collaborator

Add a composite index on quality_control_flags(run_number, detector_id) that proved to reduce computation time for GAQ summary by 50% on local.

I have a JIRA ticket

  • branch and/or PR name(s) include(s) JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected

Notable changes for users:

  • N/A

Notable changes for developers:

  • N/A

Changes made to the database:

  • N/A

Add a composite index on quality_control_flags(run_number, detector_id) that proved to reduce computation time for GAQ summary by 50% on local.
@isaachilly isaachilly self-assigned this Mar 5, 2026
@isaachilly isaachilly added the DB label Mar 5, 2026
@isaachilly isaachilly marked this pull request as ready for review March 5, 2026 16:52
@isaachilly isaachilly requested a review from graduta as a code owner March 5, 2026 16:52
Add missing CERN/ALICE GPLv3 license header to migration file.
@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 45.39%. Comparing base (cb698d6) to head (7861dfb).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...ns/v1/20260305110000-add-qcf-run-detector-index.js 60.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2099      +/-   ##
==========================================
- Coverage   45.56%   45.39%   -0.17%     
==========================================
  Files        1041     1042       +1     
  Lines       17317    17322       +5     
  Branches     3136     3136              
==========================================
- Hits         7891     7864      -27     
- Misses       9426     9458      +32     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@isaachilly isaachilly merged commit a38a180 into main Mar 6, 2026
22 of 24 checks passed
@isaachilly isaachilly deleted the improv/O2B-1539/Add-index-to-speed-up-GAQ-computation branch March 6, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants